Skip to content

Comments

WA-NEW-008: Update PasswordReset error-copying for Rails 7 errors API#632

Open
kitcommerce wants to merge 1 commit intonextfrom
wa-new-008-passwordreset-errors
Open

WA-NEW-008: Update PasswordReset error-copying for Rails 7 errors API#632
kitcommerce wants to merge 1 commit intonextfrom
wa-new-008-passwordreset-errors

Conversation

@kitcommerce
Copy link

Summary

Update User::PasswordReset#complete to iterate errors using the Rails 7 ActiveModel::Error object API (error.attribute, error.message) while maintaining backward compatibility with older Rails versions that yield [attribute, message] pairs.

Closes #625

Client impact

None expected. Internal error-handling change. No public API changes.

Verify

bundle exec ruby -Icore/test core/test/models/workarea/user/password_reset_test.rb

@kitcommerce
Copy link
Author

Dispatcher Build Gate Summary (local)

  • rubocop (diff-only): PASS (0 offenses)
  • brakeman: skipped (per repo build gate config; disabled)
  • tests (affected engines): PASS
    • core: PASS

Note: test output still prints BSON Symbol deprecation warning (expected to be addressed by WA-NEW-010 / PR #635).

@kitcommerce kitcommerce added review:architecture-pending Review in progress review:simplicity-pending Review in progress review:security-pending Review in progress review:rails-conventions-pending Rails conventions review in progress and removed review:security-pending Review in progress labels Feb 22, 2026
@kitcommerce
Copy link
Author

⚠️ Review orchestration paused: hit an LLM API rate limit while spawning Wave 1 reviewer agents (security) for this PR.

No action needed from the PR author; I’ll retry dispatch once limits clear.

@kitcommerce kitcommerce removed the review:architecture-pending Review in progress label Feb 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gate:build-passed Build gate passed review:rails-conventions-pending Rails conventions review in progress review:simplicity-pending Review in progress

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant